Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add tests #38

Draft
wants to merge 33 commits into
base: master
Choose a base branch
from
Draft

Add tests #38

wants to merge 33 commits into from

Conversation

shardros
Copy link
Member

Moves enviroment to poetry so that it is easier for new devs to get up and running.
Adds tests
Adds config for automatic linting with flakehell
Removes the calibrate camera script which doesn't work and should be supplied seperately to the robot lib

  • Workaround bug in the GG by forcing the pins to be updated in order
  • Changes to fix up default state for the GPIO now that we are setting them without direct user supervision
  • Send update command to GG in GreenGiantGPIOPinList.__init__
  • Workaround bug in the GG by forcing the pins to be updated in order
  • Changes to fix up default state for the GPIO now that we are setting them without direct user supervision
  • Send update command to GG in GreenGiantGPIOPinList.__init__
  • Add support for FlakeHell and run in CI
  • mend
  • Update FlakeHell.yml
  • use poetry
  • Beta release of AT branch
  • tests
  • update baseline.txt to have lastest errors
  • run in virtual env
  • run in virtual env
  • remove calibrate_camera

shardros and others added 30 commits November 7, 2021 11:47
To ensure that state of the GG matches the state assumed locally at init
To ensure that state of the GG matches the state assumed locally at init
Created baseline so that all of the current linter failures do not show up
To ensure that state of the GG matches the state assumed locally at init
Created baseline so that all of the current linter failures do not show up
@shardros shardros force-pushed the refactor_and_test branch from ef44009 to 0653d21 Compare June 12, 2022 16:52
@shardros
Copy link
Member Author

Squash this mess when merging to master

@shardros
Copy link
Member Author

Need to setup a brain as a CI test runner, at the very least it just needs to be a pi

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants